Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple accounts #5

Merged
merged 1 commit into from
Jan 21, 2024
Merged

Multiple accounts #5

merged 1 commit into from
Jan 21, 2024

Conversation

BrunoBernardino
Copy link
Owner

This implements a simple account switcher and a way to change/add accounts. To remove one, you have to logout from all of them.

It makes sharing family accounts easier, while keeping separate accounts/tracking per person.

This implements a simple account switcher and a way to change/add accounts. To remove one, you have to logout from all of them.

It makes sharing family accounts easier, while keeping separate accounts/tracking per person.
@BrunoBernardino BrunoBernardino self-assigned this Jan 21, 2024
@@ -1,6 +1,6 @@
.PHONY: start
start:
deno run --watch --allow-net --allow-read --allow-env main.ts
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Required for the emit update.

@@ -15,17 +15,6 @@ services:
soft: -1
hard: -1

# NOTE: This would be nice to develop with https:// locally, but it doesn't work, for whatever reason, so we need a system caddy instead
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would never work "easily" (only if the deno piece was also running in docker instead of the host machine, and even still, there are other certificate issues), so I'm removing it.

@BrunoBernardino BrunoBernardino merged commit 713f5fd into main Jan 21, 2024
1 check passed
@BrunoBernardino BrunoBernardino deleted the feature/multiple-accounts branch January 21, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant